Skip to content

Develop#45

Merged
DannyNs merged 6 commits into
mainfrom
develop
Jun 18, 2026
Merged

Develop#45
DannyNs merged 6 commits into
mainfrom
develop

Conversation

@DannyNs

@DannyNs DannyNs commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Daniel Kutyła and others added 6 commits June 17, 2026 12:55
The chat MCP module exposes 11 tools, but README and the managed
CLAUDE.md template only listed the 5 core chat tools. Add the missing
chat_status plus the pipe subsystem (pipe_submit, pipe_read_output,
pipe_get_assignment, pipe_list_assignments, pipe_status) and explain
multi-stage pipe orchestration (linear, fan-out, synthesizer).
resolveDefaultShell() returned a bare 'bash' when $SHELL was unset.
Under a daemon/GUI launch (devglide start) the process env can lack
PATH, so node-pty's posix_spawnp could not locate the bare command and
failed with 'posix_spawnp failed' when creating a shell pane on macOS.

- pickUnixShell(): resolve an absolute shell (/bin/zsh, /bin/bash,
  /bin/sh, ...), never a bare name. Mirrors the Windows branch.
- withDefaultPath(): guarantee a sane PATH in the spawn env so commands
  run inside the shell even when the parent env has none.

Both extracted as pure helpers with unit coverage in shell-config.test.ts.
devglide setup runs scripts/build-mcp.mjs, which bundled the voice MCP
with esbuild. nodejs-whisper (an optional whisper.cpp STT dependency) was
not in the external allowlist, so esbuild tried to inline it. When the
package is absent or fails to install, the build failed with 'Could not
resolve nodejs-whisper' and aborted every remaining bundle, degrading all
servers to the tsx fallback.

The voice provider already imports nodejs-whisper lazily and degrades
gracefully at runtime, so it should be external like better-sqlite3 and
node-pty rather than bundled.
fix(shell): spawn shell via absolute path on macOS/Linux
docs(chat): document chat_status and pipe orchestration tools
…r-external

fix(build): mark nodejs-whisper external in MCP bundling
@DannyNs DannyNs merged commit 6466f4f into main Jun 18, 2026
1 check passed
@DannyNs DannyNs deleted the develop branch June 18, 2026 10:29
@DannyNs DannyNs restored the develop branch June 18, 2026 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant